Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Separate out precommit configuration #809

Merged
merged 1 commit into from
Jan 22, 2024
Merged

Separate out precommit configuration #809

merged 1 commit into from
Jan 22, 2024

Conversation

hanbyul-here
Copy link
Collaborator

Fixed the problem (ts check does not scope the type check to the staged files. ex. throwing module not found error if the module is imported in a staged file, but not added to the stage) of ts-check in precommit hook by following: lint-staged/lint-staged#825 (comment)

Copy link

netlify bot commented Jan 22, 2024

Deploy Preview for veda-ui ready!

Name Link
🔨 Latest commit 704a339
🔍 Latest deploy log https://app.netlify.com/sites/veda-ui/deploys/65aec2e140c46e0008e3f631
😎 Deploy Preview https://deploy-preview-809--veda-ui.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@stephenkilbourn
Copy link
Contributor

great catch! 🧠

@hanbyul-here hanbyul-here merged commit 980d209 into main Jan 22, 2024
8 checks passed
@hanbyul-here hanbyul-here deleted the precommit-ff branch January 22, 2024 20:01
sandrahoang686 added a commit that referenced this pull request Feb 13, 2024
https://github.com/NASA-IMPACT/veda-ui/releases/tag/v4.1.0

## 🎉 Features
- Experimental CMR Layer:
#805

## 🚀 Improvements
- Style 
- Documentation
- Rename critical error page:
#806
- Add back button on Exploration page:
#779
- Add ts, lint check for precommit hooks:
#788,
#809
- "skip to main" button for navigation:
#808

## 🐛 Fixes
- Return datasets even when there is a dataset without summaries:
#786
- Show all the datasets on Data Catalog page:
#837
- Block Map user defined position fix:
#784
- Geocoder centering on various projecctions:
#826
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants